-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
devsecops/emvaldes/remote-branch #17022
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor nitpicks around SHA pinning, GTG otherwise.
it's already addressed as the result of what was requested in the review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see/resolve inline comments
This issue was already addressed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. If the tag is 1.0.1,, then please merge.
This PR imports the GitHub Repository (Action): remote-branch
This PR does not fix any currently exploitable issue.
Checklist
Process
Specific Security-related subjects a reviewer should pay specific attention to
Does this PR introduce new endpoints? No
Does this PR include changes in authentication and/or authorization of existing endpoints? No
Does this change introduce new dependencies that need vetting? No
Does this change require changes to our infrastructure? No
Does logging contain sensitive data? No
Does this PR include or remove any sensitive information itself? No
What are the potential security threats and mitigations? N/A
Have you ensured logging does not contain sensitive data? N/A
Have you received any additional approvals needed for this change? N/A